Matt Turner <mattst88@gmail.com>: Author Summary

Builds triggered by Matt Turner <mattst88@gmail.com>

Builds triggered by an author are those builds which contains changes committed by the author.
1
0 (0%)
1 (100%)

Breakages and fixes

Broken means the build has failed but the previous build was successful.
Fixed means that the build was successful but the previous build has failed.
0 (0% of all builds triggered)
0 (0% of all builds triggered)
0
Build Completed Code commits Tests
SP › SPTD › #15 1 year ago
Fix set but unused warnings
su_proxy.c: In function ?forwarder_accept?:
su_proxy.c:361:7: warning: variable ?events? set but not used [-Wunused-but-set-variable]
  361 |   int events;
      |       ^~~~~~
Fix sequence point warning
test_memmem.c: In function ?test_notfound?:
test_memmem.c:97:19: warning: operation on ?a? may be undefined [-Wsequence-point]
   97 |   TEST_P(memmem(a = "a\0bc", 4, "a\0bc", 4), a);
      |                 ~~^~~~~~~~~
Fix buffer size warning
torture_url.c: In function ?test_sip?:
torture_url.c:368:45: warning: ?%s? directive output may be truncated writing up to 511 bytes into a region of size 498 [-Wformat-truncation=]
  368 |     snprintf(pu, sizeof(pu), "sip:test@host;%s", param);
      |                                             ^~   ~~~~~
torture_url.c:368:5: note: ?snprintf? output between 15 and 526 bytes into a destination of size 512
  368 |     snprintf(pu, sizeof(pu), "sip:test@host;%s", param);
      |     ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
Remove unused variables
torture_sresolv.c: In function ?test_api_errors?:
torture_sresolv.c:142:7: warning: variable ?s? set but not used [-Wunused-but-set-variable]
  142 |   int s, fd;
      |       ^
torture_sresolv.c:177:5: warning: ?sockets? may be used uninitialized [-Wmaybe-uninitialized]
  177 |   s = sockets[0];
      |   ~~^~~~~~~~~~~~
torture_sresolv.c:143:7: note: ?sockets? declared here
  143 |   int sockets[20];
      |       ^~~~~~~
Fix -Wunused-value warnings
stun_internal.h:90:66: warning: right-hand operand of comma expression has no effect [-Wunused-value]
   90 |   SU_DEBUG_5(("%s: %s: %s\n", __func__, #what, su_strerror(err))), \
      |                                                                  ^
stun.c:1318:5: note: in expansion of macro ?STUN_ERROR?
 1318 |     STUN_ERROR(one, SO_ERROR);
      |     ^~~~~~~~~~
Fix array size
check_sres_sip.c: In function ?resolver_setup?:
check_sres_sip.c:113:19: warning: array subscript 2 is above array bounds of ?su_addrinfo_t[2]? {aka ?struct addrinfo[2]?} [-Warray-bounds]
  113 |   hint_udp_tcp_tls[2].ai_protocol = TPPROTO_TLS;
      |   ~~~~~~~~~~~~~~~~^~~
check_sres_sip.c:69:22: note: while referencing ?hint_udp_tcp_tls?
   69 | static su_addrinfo_t hint_udp_tcp_tls[2];
      |                      ^~~~~~~~~~~~~~~~
check_sres_sip.c:114:19: warning: array subscript 2 is above array bounds of ?su_addrinfo_t[2]? {aka ?struct addrinfo[2]?} [-Warray-bounds]
  114 |   hint_udp_tcp_tls[2].ai_next = NULL;
      |   ~~~~~~~~~~~~~~~~^~~
check_sres_sip.c:69:22: note: while referencing ?hint_udp_tcp_tls?
   69 | static su_addrinfo_t hint_udp_tcp_tls[2];
      |                      ^~~~~~~~~~~~~~~~

Fixes: https://github.com/freeswitch/sofia-sip/issues/89
Use memmove for overlapping src and destination
In file included from /usr/include/string.h:519,
                 from stun.c:41:
In function ?memcpy?,
    inlined from ?stun_test_nattype? at stun.c:1251:5:
/usr/include/bits/string_fortified.h:29:10: warning: ?__builtin_memcpy? accessing 32 bytes at offsets 96 and 72 overlaps 8 bytes at offset 96 [-Wrestrict]
   29 |   return __builtin___memcpy_chk (__dest, __src, __len,
      |          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   30 |                                  __glibc_objsize0 (__dest));
      |                                  ~~~~~~~~~~~~~~~~~~~~~~~~~~
In function ?memcpy?,
    inlined from ?stun_test_lifetime? at stun.c:2740:5:
/usr/include/bits/string_fortified.h:29:10: warning: ?__builtin_memcpy? accessing 32 bytes at offsets 96 and 72 overlaps 8 bytes at offset 96 [-Wrestrict]
   29 |   return __builtin___memcpy_chk (__dest, __src, __len,
      |          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   30 |                                  __glibc_objsize0 (__dest));
      |                                  ~~~~~~~~~~~~~~~~~~~~~~~~~~
Fix pointer memaccess warning
test_sresolv.c: In function ?test_deinit?:
test_sresolv.c:1875:25: warning: argument to ?sizeof? in ?memset? call is the same expression as the destination; did you mean to dereference it? [-Wsizeof-pointer-memaccess]
 1875 |   memset(ctx, 0, sizeof ctx);
      |                         ^~~
Fix misleading indentation
Fix -Wunused-value warnings
torture_sresolv.c:342:13: warning: right-hand operand of comma expression has no effect [-Wunused-value]
  342 |   for (i = 0, N; i < N; i++)
      |             ^
Remove unused function
test_sresolv.c: In function ?fill_stack?:
test_sresolv.c:1920:9: warning: variable ?array? set but not used [-Wunused-but-set-variable]
 1920 |   int i,array[32768];
      |         ^~~~~
Testless build
Build Completed Code commits Tests
Build Completed Code commits Tests